wayland: Unset button modifiers on pointer enter

Due to implicit grabs, we basically can guarantee that the pointer
won't have any buttons pressed at the time of wl_pointer.enter.
Seems like a good place to unset any button modifiers that might
have been left stale by compositor grabs.

wayland: Implement DnD actions as per wl_data_device v3

Implement as a managed GdkDragContext, which actually fits nicely
with the drag-and-drop model in wayland.

wayland: Add internal gdk_wayland_seat_set_global_cursor()

This can be used for cases (like DnD) where there isn't necessarily
a grab, but we want a global pointer cursor.

gtkdnd: Reset widget controllers when starting DnD.

Otherwise there's no full guarantees that this will happen.

gtk: Add private gtk_widget_reset_controllers()

This is a one-shot call that can be called on the places we
know we're taking input away.

gtkdnd: Optionally use gdk_drag_context_manage_dnd()

When this is in use, there's essentially a bunch of dead code here.
When all backends are ported, we'll be able to remove grab/cursor
management plus a bunch of source-side event handlers.

x11: Implement gdk_drag_context_manage_dnd()

This includes managing input events and source-side DND events,
as well as setting the appropriate cursor and emitting the signals
that are expected in this mode of operation.

gdk: Add gdk_drag_get_cursor()

This function (most similar to gtk_drag_get_cursor() helps figure out
the right cursor that applies to a given action. To be used by the
various backends.

gdk: Run DnD internal handlers before the main event handler

We'll be stealing those to GTK+, if the GdkDragContext manages
the DnD operation.

gdk: Allow internal management of source-side DnD

We've traditionally left GTK+ to handle the input side of things,
letting GDK only manage the windowing-specific messaging. This
way of splitting responsibilities is not compatible however with
some backends, we must fold then input management at the DnD stage
into GDK (and backends) domain.

The gdk_drag_context_manage_dnd() call is meant to be the entry
point for this mode of operation, if the drag and drop operation
becomes managed, the caller (i.e. gtkdnd.c) doesn't need to perform
grabs, nor manage input events itself.

As a consequence of this, different aspects now belong to the
backend GdkDragContext implementation:
- Because the caller doesn't see keyboard events anymore,
  keyboard navigation must be managed in GDK, so is the decision
  of the current action based on modifiers/button pressed.
- Because the caller won't see input events in general, the lifetime
  of the drag and drop operation is now communicated through the
  ::drop-performed, ::dnd-finished and ::cancel events
- Because the caller doesn't participate anymore on the action
  being chosen, the pointer cursor must be set by the backend.
  The caller is rather notified of the final action through the
  ::action signal.

The caller is still responsible of dealing with the corresponding
GdkSelection, ensuring its ownership and communicating the supported
mimetypes.

widget: Call gdk_window_mark_paint_from_clip() again

The proper window to call it is the event window, as the call itself
ignores non-native windows anyway.

Fixed typo childen->children

gtk-demo: Add a progress bar to foreigndrawing

This is another commonly requested widget.

container: Properly reorder no-window children

... that are setup with gtk_widget_set_parent_window().

Fixes scrollbars not being drawn in GtkScrolledWindow.

widget: Add forgotten push_group code

... and remove the also forgotten void function that lingered around
with it.

Fixes opacity=0 parts like inactive spinners or sort indicators in
treeview headers being drawn since last commit.

Oops.

widget: Redo drawing code

Previously, we had a special cae to draw subwindows of widgets.

This is not necessary as conformant widgets should be able to properly
render themselves when all windows need to be painted.
From now on assume that is the case.

We therefore paint nonnative GDK windows "inline" by just returning TRUE
for gtk_cairo_should_draw_window() for those windows.

This speeds up hilighting different rows in the listbox gtk-demo example
tremendously (by a factor of 10 or more) as the previous code was
O(<number of non-window subwidgets> *
<number of subwindows>) which in the listbox example were ~15,000 and
~2,000 respectively.

actionbar: Don't forall() widgets twice

When using forall(), only list the revealer, which lists the box
containing all the children. When using foreach(), bypass revealer and
box and list all children added to the box.

Updates

text view: Expose cursor blink control

Derived classes like GtkSourceView with their own ::key-event
handler need access to this, in order to make their keynav
as nice as the builtin one, wrt to caret visibility.

https://bugzilla.gnome.org/show_bug.cgi?id=760748

x11: Set event->scroll.is_stop

We still figure this out from 0/0 scroll events. This method is
not intended to last forever, but it's something we can cling to
so far.

https://bugzilla.gnome.org/show_bug.cgi?id=756729

wayland: add gdk_event_is_scroll_stop_event()

And use it to handle kinetic scrolling in the GtkScrolledWindow.

However, dropping the delta check causes the X11-based kinetic
scroll to break since we don't have the stop event here. Correct handling of
xf86-input-libinput-based scroll events is still being discussed.

https://bugzilla.gnome.org/show_bug.cgi?id=756729

wayland: add support for wl_pointer frame/axis_source/axis_discrete/axis_stop

This adds support for the new wl_pointer events available in v5.

The wl_pointer.axis_source events can be ignored for the purposes here, the
main reason they exist is so that the combination of axis_source=finger and
axis_stop triggers kinetic scrolling. We don't need to care about the source,
axis_stop is enough for us to tell us when we're scrolling.

The wl_pointer.frame events group events together and is intended as a
mechanism to coalesce events together. This for example allows us to now
send a single GTK scroll event for a diagonal scroll. Previously, the two
wl_pointer.axis events had to be handled separately.

The wl_pointer.axis_discrete event sends mouse wheel clicks where
appropriate, and is translated into up/down/left/right scroll events.

https://bugzilla.gnome.org/show_bug.cgi?id=756729

Drop a reference to regions

Regions don't work anymore, so don't recommend them in the docs.

im context simple: Protect shared data structures

Since a41f02f9b1843e0f0085f801430e55f413a9bf9c, GtkIMContextSimple
uses threads to load X Compose files. It does that every time a new
im context object is initialized, so we can easily end up with multiple
threads accessing the shared global_tables list at the same time.

Use a lock to prevent that.

https://bugzilla.redhat.com/show_bug.cgi?id=1276432

cssnode: Don't leak the style cache on destruction

gdkdisplay-x11: Plug memory leak

text view: Update selection node visibility

Make the node invisible if we don't have a selection.

entry: Expand the css node docs

Mention the context menu.

text view: Expand CSS node docs

Mention the selection node, and the context menu.

Forgotten file

notebook: Don't crash when dnd'ing tabs out

https://bugzilla.gnome.org/show_bug.cgi?id=760754

accellabel: Remove unused variable

gdk: Clarify docs about scale

Explicitly state where we return application pixels as opposed
to device pixels.

Trivial typo fix

Add more CSS node tests for notebooks

entry: Deprecate shadow-type

Things like that should be handled using css. Also, the property value
is unused inside GtkEntry.

W32: Detect ticks wraparound

https://bugzilla.gnome.org/show_bug.cgi?id=748327

Add CSS node tests for checkbutton rtl flipping

Add css node tests for expander rtl flipping

expander: Flip the alignment of the vertical box

To ensure that the title moves to the other side as expected
in RTL, use GTK_ALIGN_START/END instead of GTK_ALIGN_FILL
when packing the title gadget into the vertical box, and
flip the alignment when the text direction changes.

expander: Handle rtl changes again

Now that the box gadget has api for it, use it here to
change the order of the gadgets for right-to-left.

checkbutton: Handle rtl changes again

Now that the box gadget has api for it, use it here to
change the order of the gadgets for right-to-left.

box gadget: Add a function to reverse children

This can be used to handle text direction changes.

reftests: Fix up for easy CSS changes

This is mostly search and replace ala
  GtkButton => button
  GtkWindow => window
  .button => button
or removing style properties that aren't used anymore like
  -GtkButton-default-border: 0

reftests: Remove test

We no longer match subtypes, so remove the test for it.

checkbutton: Use a box gadget

Now that builtin icons have a baseline, we can just use
a box gadget with baseline alignment for the indicator
and the label.

builtin icon: set a baseline

Most uses of builtin icons (check and radio buttons,
expanders, etc) are placed next to labels, so they should
be properly positioned wrt to the baseline. Lacking anything
better, give the builtin icons a baseline that places the
center of the icon at the strikethrough position.

box gadget: Deal with optional arguments

The baseline arguments to the preferred size callback can
be NULL. Don't crash if they are.

Trivial formatting fix

Document restrictions on gtk_style_context_get

This makes it official that passing a different state
is not generally supported in the getters for CSS properties.

notebook: show tabs when asked to

gtk_notebook_set_show_tabs must make the header gadget
visible when show_tabs is TRUE, not the other way around.

https://bugzilla.gnome.org/show_bug.cgi?id=760596

accel label: Fix RTL handling

The 'mad hack' that GtkAccelLabel used to affect the GtkLabel
draw function broke with the introduction of gadgets, since
the positioning is no longer relative to the widgets' allocation
at the time of the call, but rather to the gadgets allocation.

Instead of coming up with an even madder hack to keep this
working, give the GtkLabel draw function knowledge about accel
labels.

https://bugzilla.gnome.org/show_bug.cgi?id=760663

inspector: Make cursor-size combo wider

Cursor sizes typically have 2 digits, lets not clip them.

Expand window style class documentation a bit

Adwaita: Keep entry progress visible

When the entry is focused, make the progress one pixel wider
so it doesn't get swallowed by the focus outline.

stylechange: Do the right thing when old and new style are equal

We don't need to compare all their CSS values to figure out that nothing
changed. We know that.

cssnode: Redo GTK_DEBUG=no-css-cache

Previously this setting would just invalidate the whole CSS tree and
thereby hopefully avoid all cache usage.

Now, we actually don't cause extra invalidations anymore, but instead
avoid ever inserting anything into the cache when this setting is set.

cssnode: Implement the style cache

This essentially copies the previous cache implementation. With one
caveat: It is now attached to and maintained by the CssNode, not by the
CssStyle.

And this is important because styles may be reused in incompatible
situations which would cause cache collisions and lead to broken CSS in
weird situations.

cssnode: Split out the style cache

For now, the split out style cache doesn't cache anything. This is
mostly to make sure that bisections of wrong caching behavior will
bisect down to the commit that actually adds caching.

Adwaita: stop axing toolbar separators

Adwaita: style textview derived widgets

color scale: Improve themability

Add the .color class, so themes can style scale.color.
Clip the trough rendering to the content path, so that
themes can set rounded corners.

treeview: Use foreground color to draw dnd marker

This is still not perfect, but makes the dnd marker better visible
in HighContrastInverse.

frame: Don't overlap label and child

We didn't do this previously, so don't start now.

https://bugzilla.gnome.org/show_bug.cgi?id=760528

Adwaita: combo style cleanup and fixes

fixes https://bugzilla.gnome.org/show_bug.cgi?id=760524

combo: Add .linked class when appropriate

Add the .linked style class to combo boxes which have
an entry, since that is what they are.

combobox: Document the popup window CSS node

application: Protect against double shutdown

It is not something that can happen normally.
But if it does, we shouldn't crash.

https://bugzilla.gnome.org/show_bug.cgi?id=760680

mir: add default GdkSeat for the default GdkDisplay

widget: remove queue_resize in set_sensitive

Calling gtk_widget_propagate_state is enough to queue a resize if
needed

See da7a4089fe82981d1412aaa295e76d47a81ad6df

wayland: Fix an endless loop

Adwaita: remove messagedialog label squashing hack

since Matthias fixed this properly.

message dialog: Prevent empty title from being shown

We don't want to grow the titlebar by showing an empty label.

Adwaita: some dialog love

addresses https://bugzilla.gnome.org/show_bug.cgi?id=760525

expander: Destroy gadgets after child widgets

With child widgets now being sorted into box gadgets, bad things
happen if those gadgets disappear prematurely.

notebook: Destroy gadgets after chaining up

While chaining up, we might still call into the notebook code. And that
code expects the gadgets to be there.

https://bugzilla.gnome.org/show_bug.cgi?id=760640

expander: Convert to use box gadgets

Use a vertical box gadget for the overall expander, and a
horizontal one for the title row. This lets us get rid of
all the custom allocation code here.

expander: Deprecate spacing property

The same can be achieved with margins on the child, and
the spacing property gets in the way of converting the
expander to use box gadgets.

box gadget: Implement cross-axis alignment

So far, the box gadget is always allocating all children the
full size in the cross axis. This behavior corresponds to the
align-items: stretch behavior in
https://www.w3.org/TR/css-flexbox-1/#align-items-property
This commit implements the other modes described there.

While widgets have halign/valign properties that we can use for
this, the API for inserting gadgets has to change to take an
extra align parameter. All callers have been updated to pass
GTK_ALIGN_FILL, since that corresponds to the previous behavior.

https://bugzilla.gnome.org/show_bug.cgi?id=760668

Updated POTFILES.skip

win32: get rid of _gdk_screen global variable

The screen is part of the display so it should be belong to it.

win32: fix warning

Windows library manager fix

The Makefile.am's had a dependency on incorrectly named libtool
archives for the generation of the .lib files.

https://bugzilla.gnome.org/show_bug.cgi?id=760388

gtkwidgetpath: update gtk_widget_path_to_string

From gtk_widget_path_iter_set_object_name documentation:
"When set, the object name overrides the object type when matching CSS"

Update gtk_widget_path_to_string to match this behaviour.

Fix typo in previous documentation fix for gtk_css_provider_load_from_data()

Document -gtk-recolor syntax

Add a way to load recolored icons in CSS

This can come in handy to load images that are not in an icon theme,
but still benefit from symbolic recoloring.

https://bugzilla.gnome.org/show_bug.cgi?id=760536

icon theme: Export recoloring function

This will be used in subsequent commits.

https://bugzilla.gnome.org/show_bug.cgi?id=760536

icon theme: Add a way to create GtkIconInfos from files

This is sometimes needed, and calling into actual icon theme
code just for it is confusing - the resulting icon does not
depend on the icon theme at all.

https://bugzilla.gnome.org/show_bug.cgi?id=760536

Adwaita: pathbar style cleanup

cssstyleproperty: Expose min-width/height

Adwaita: remove superfluous selector on keycap styling

now that keys are made directly with labels istead of frames.

boxgadget: Don't use g_assert

Use g_return_if_fail instead so nothing abort()s.

entry: Don't draw icons twice

entry: Remove unused code

label: Remove some unneeded conditionals

We are never passing NULL here.

shortcutlabel: Use a label directly

We can everything we need without the frame around it now.

GDK W32: Don't use WM-drawn shadow for tooltips

This prevents WM from drawing shadows around tooltip windows,
which, in Adwaita, should have no shadow and are CSD-ish (which means
that tooltip window is larger than it looks, and WM draws the shadow
only on the outside, leaving a gap between the visible tooltip edge and
the shadow).

https://bugzilla.gnome.org/show_bug.cgi?id=759898

win32: remove _gdk_root global variable

Ensure that the fstat fix is not applied on x86_64